Skip to content

ES-2979 - project config resolution#219

Merged
jgrantr merged 4 commits intomasterfrom
feature/ES-2979-project-config-local-resolution
Apr 2, 2026
Merged

ES-2979 - project config resolution#219
jgrantr merged 4 commits intomasterfrom
feature/ES-2979-project-config-local-resolution

Conversation

@jgrantr
Copy link
Copy Markdown
Contributor

@jgrantr jgrantr commented Apr 1, 2026

  • can now resolve project config even without just a secret or a environment variable
  • adds SSM, stack, and CF resolvers
  • uses same established patterns to make an async call synchronous.

Note

Medium Risk
Medium risk because it changes how runtime configuration is discovered and resolved, including new AWS API calls (SSM/STS/CloudFormation) and stricter failures for unresolved templates, which could break existing deployments if env vars/configs are incomplete.

Overview
Improves configuration resolution by auto-discovering a local project-config.def.json when RSF_CONFIG isn’t set (and explicitly erroring in Lambda to avoid unsupported discovery), and by adding a Stage (capitalized) keyword alongside stage for template substitution.

Adds new resource resolvers for ssm, cf (CloudFormation exports), and stack (pseudo-parameters like AWS::AccountId via STS and AWS::Region), with env-var overrides, caching, and validation that template variables are fully resolved.

Extends aws-sdk-sync to support additional AWS clients (SSM, STS, CloudFormation) and allows overriding the AWS SDK package name when invoking a service, plus makes Secrets Manager resolution tolerate non-JSON secrets.

Written by Cursor Bugbot for commit 9dfaf8f. This will update automatically on new commits. Configure here.

- can now resolve project config even without just a secret or a environment variable
- adds SSM, stack, and CF resolvers
- uses same established patterns to make an async call synchronous.
@ch-snyk-sa
Copy link
Copy Markdown

ch-snyk-sa commented Apr 1, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@jgrantr jgrantr merged commit 9704fba into master Apr 2, 2026
8 checks passed
@jgrantr jgrantr deleted the feature/ES-2979-project-config-local-resolution branch April 2, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants